Manually update to idl@3.77.0#2482
Conversation
|
Thanks for the PR! This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged. |
| }; | ||
|
|
||
| interface HTMLOrSVGElement { | ||
| interface HTMLOrSVGOrMathMLElement { |
There was a problem hiding this comment.
The old name should be preserved somehow, right?
There was a problem hiding this comment.
I don't have proof that this is actively extended out there, but maybe...
There was a problem hiding this comment.
I am pretty sure that HTMLOrSVGElement is used in many code bases like protected __element: HTMLOrSVGElement or public addElement(element: HTMLOrSVGElement).
So this type should be kept valid
There was a problem hiding this comment.
Pushed a fix, does it look good enough? 👀
There was a problem hiding this comment.
Seems fine? Not sure about the impl itself but it has the effect I was looking for
3 participants
Closes #2481
Closes #2467